add skill to find and improve alt text for pkgdown documentation#47
add skill to find and improve alt text for pkgdown documentation#47gadenbuie merged 8 commits intoposit-dev:mainfrom
Conversation
gadenbuie
left a comment
gadenbuie
left a comment
There was a problem hiding this comment.
Thanks! This looks great. My biggest comment is around considering merging this skill with the quarto alt text skill, which I think would be a good idea.
|
|
||
| ```r | ||
| # Build only articles (faster than full site) | ||
| Rscript -e "pkgdown::build_articles()" |
There was a problem hiding this comment.
I was surprised this wasn't picked up by pkgdown::check_pkgdown() or pkgdown::pkgdown_sitrep(). Might be worth opening an issue in pkgdown to see if there could be a faster path.
There was a problem hiding this comment.
I have an PR for this here: r-lib/pkgdown#2985
|
Okay, did a bunch. Now there is a new skill I also added a PR to pkgdown r-lib/pkgdown#2985, to have it give better feedback on missing alt-text. however since that is not merged or on CRAN yet I have left the instructions to grep through instead |
| Quarto documents and pkgdown sites. | ||
|
|
||
| Generate accessible alt text for data visualizations in this project. | ||
| If you have the `quarto` plugin installed, reinstall it to get the updated skill: |
There was a problem hiding this comment.
I might be wrong, but I think we should just move the skill. I don't think there's a good migration story for skills/plugins/etc. Or at least, we've moved skills around before and haven't heard much yelling.
OTOH if you have a clear vision for how this works and is better than just moving the skill, let me know :)
There was a problem hiding this comment.
sounds good. I didn't have any major plans, so i just moved the skill now
| name: alt-text | ||
| description: > | ||
| Generate and improve accessible alt text for data visualizations and images | ||
| in R packages and Quarto documents. Use when the user wants to add, improve, |
There was a problem hiding this comment.
It'd be nice to have a similar skill for Shiny apps, too. If you wanted to add that, I'd be grateful. Or I'd be happy to just open an issue and come back to it when I have a minute.
There was a problem hiding this comment.
I don't know enough about shiny to write a good skill for that :)
gadenbuie
left a comment
gadenbuie
left a comment
There was a problem hiding this comment.
Thanks @EmilHvitfeldt!
2 participants
Pretty much what the title says.
It is very similarly scoped to the quarto-alt-text skill. It is instructed to read nearby text for context, and follow the same rules. the difference is how it finds the images and where to place the alt text
Evaluation